Skip to content

Finalize SCAI portfolio presentation - #3

Merged
daetan999 merged 1 commit into
mainfrom
agent/finalize-readme-showcase
Aug 5, 2026
Merged

Finalize SCAI portfolio presentation#3
daetan999 merged 1 commit into
mainfrom
agent/finalize-readme-showcase

Conversation

@daetan999

Copy link
Copy Markdown
Owner

What changed

  • replaces the repository landing page with a hiring-manager-first README
  • adds a deliberate SVG banner, five reproducible UI captures, and a 10.3-second fixture-backed demo GIF
  • documents the three-stage design, deterministic arithmetic boundary, formulas, defaults, batch contract, limitations, and deployment path
  • adds a Playwright/Pillow capture harness with schema-validated recorded fixtures and no model calls
  • adds a GitHub Actions cost-test workflow with an 80% coverage gate
  • fixes numeric parsing in the assumptions UI so $3.75 per 1,000 inferences renders as 3.75, not 3.751
  • updates Pillow to 12.3.0 after the final dependency audit identified advisories against 12.0.0

Why

SCAI is a portfolio artifact for AI-infrastructure hiring managers and solutions architects. The repository now makes the core design decision legible in seconds: Gemini retrieves evidence and drafts language, while deterministic Python owns every number and refuses unverified pricing.

Validation

  • 8 tests passed
  • app/cost.py coverage: 93%
  • dependency audit: no known vulnerabilities
  • secret-pattern scan: clean
  • all five screenshots reproduced at a 1440×900 viewport with 2× device scale
  • demo GIF: 10.3 seconds and 349 KB
  • batch endpoint and documented CSV schema smoke-tested without model calls
  • git diff --check passed

Environment-bound checks

Docker and gcloud are unavailable locally, so container build/run and Cloud Run deployment remain documented but were not reported as successful local checks.

@daetan999
daetan999 merged commit bfbcd75 into main Aug 5, 2026
2 checks passed
@ecc-tools

ecc-tools Bot commented Aug 5, 2026

Copy link
Copy Markdown

Analyzing 200 commits...

1 similar comment
@ecc-tools

ecc-tools Bot commented Aug 5, 2026

Copy link
Copy Markdown

Analyzing 200 commits...

@ecc-tools

ecc-tools Bot commented Aug 5, 2026

Copy link
Copy Markdown

Analysis Complete

Generated ECC bundle from 1 commits | Confidence: 50%

View Pull Request #4

Repository Profile
Attribute Value
Language Python
Framework Not detected
Commit Convention conventional
Test Directory separate
Changed Files (14)
Metric Value
Files changed 14
Additions 530
Deletions 91

Top hotspots

Path Status +/-
scripts/capture.py added +243 / -0
README.md modified +98 / -90
scripts/fixtures/capture.json added +93 / -0
tests/test_capture.py added +45 / -0
.github/workflows/tests.yml added +24 / -0

Top directories

Directory Files Total changes
scripts 1 243
. 2 191
scripts/fixtures 1 93
tests 1 45
.github/workflows 1 24
Analysis Depth Readiness (evidence-backed, 50%)

ECC Tools uses this to decide whether recommendations should stay at commit-history/setup guidance or expand into CI, security, harness, reference-set, AI-routing, and team backlog work.

Area Status Evidence / Next Step
Commit history Partial 1 commits sampled
CI/CD signals Ready .github/workflows/tests.yml
Security evidence Missing Add AgentShield, audit, SARIF, SBOM, or security review evidence so recommendations can cover security posture.
Harness configuration Missing Add Claude, Codex, OpenCode, Zed, dmux, MCP, plugin, or cross-harness config evidence for harness-agnostic recommendations.
Reference/eval evidence Ready scripts/fixtures/capture.json
AI routing and cost controls Ready docs/03-cost.png
Team handoff and project tracking Missing Add roadmap, runbook, project, Linear, or follow-up tracking docs so generated work can land in a team queue.
Reference Set Readiness (0/7, 0%)
Area Status Evidence / Next Step
Deep analyzer corpus Missing Add analyzer fixture, golden, benchmark, or reference-set files that can catch analyzer regressions.
RAG/evaluator comparison Missing Add retrieval or evaluator reference-set comparison fixtures with expected ranking behavior.
PR salvage/review corpus Missing Add stale-PR, review-thread, reopen-flow, or salvage reference cases for queue cleanup automation.
Discussion triage corpus Missing Add public discussion triage fixtures, golden cases, or reference sets for informational, answered, and no-response classifications.
Harness compatibility Missing Add cross-harness, adapter-compliance, or harness-audit evidence for Claude, Codex, OpenCode, Zed, dmux, and agent surfaces.
Security evidence Missing Attach security evidence such as SBOMs, SARIF, audit reports, or AgentShield evidence packs.
CI failure-mode evidence Missing Add captured CI failure logs, dry-run fixtures, or troubleshooting docs for common workflow failure modes.
Likely Future Issues (2)
Severity Signal Why it may show up
MEDIUM CI workflow changes may ship without failure-mode evidence 1 CI/test-runner paths changed; 0 CI failure-mode evidence artifacts changed
MEDIUM Dependency or CI drift could surface after merge CI/workflow files changed; no lockfile changes detected
  • CI workflow changes may ship without failure-mode evidence: The PR changes CI workflows or test-runner entrypoints without touching CI failure fixtures, captured logs, troubleshooting notes, or regression evidence.
  • Dependency or CI drift could surface after merge: Package or workflow changes landed without an accompanying lockfile update, which often turns into CI or release noise later.
Suggested Follow-up Work (2)
Type Suggested title Targets
PR ci: add failure-mode evidence for .github/workflows/tests.yml .github/workflows/tests.yml
PR chore: refresh lockfile and validate CI after dependency updates .github/workflows/tests.yml
  • ci: add failure-mode evidence for .github/workflows/tests.yml: Backfill CI failure-mode evidence before another workflow or test-runner change lands on the touched surface.
  • chore: refresh lockfile and validate CI after dependency updates: Package or workflow changes without a lockfile refresh tend to turn into noisy follow-up fixes after merge.

Copy-ready bodies

ci: add failure-mode evidence for .github/workflows/tests.yml

## Summary
- Add CI failure-mode evidence for the recently changed workflow or test-runner surface.

## Why
- Backfill CI failure-mode evidence before another workflow or test-runner change lands on the touched surface.

## Touched paths
- `.github/workflows/tests.yml`

## Validation
- Add or update a CI failure fixture, captured failing log, troubleshooting note, workflow dry-run evidence, or regression test for the changed CI/test-runner behavior.
- Run the affected workflow or test-runner entrypoint locally or in CI and record pass/fail evidence.

chore: refresh lockfile and validate CI after dependency updates

## Summary
- Refresh the lockfile and rerun CI after the dependency or workflow changes in this PR.

## Why
- Package or workflow changes without a lockfile refresh tend to turn into noisy follow-up fixes after merge.

## Touched paths
- `.github/workflows/tests.yml`

## Validation
- Refresh the lockfile in the same package manager used by the repo.
- Run the repo typecheck / test / CI entrypoints that depend on the updated package graph.
Generated Instincts (6)
Domain Count
git 2
code-style 3
testing 1

After merging, import with:

/instinct-import .claude/homunculus/instincts/inherited/SCAI-instincts.yaml

Files

  • .claude/ecc-tools.json
  • .claude/skills/SCAI/SKILL.md
  • .agents/skills/SCAI/SKILL.md
  • .agents/skills/SCAI/agents/openai.yaml
  • .claude/identity.json
  • .codex/config.toml
  • .codex/AGENTS.md
  • .codex/agents/explorer.toml
  • .codex/agents/reviewer.toml
  • .codex/agents/docs-researcher.toml
  • .claude/homunculus/instincts/inherited/SCAI-instincts.yaml

ECC Tools | Everything Claude Code

@ecc-tools

ecc-tools Bot commented Aug 5, 2026

Copy link
Copy Markdown

Analysis Complete

Generated ECC bundle from 1 commits | Confidence: 50%

View Pull Request #5

Repository Profile
Attribute Value
Language Python
Framework Not detected
Commit Convention conventional
Test Directory separate
Changed Files (14)
Metric Value
Files changed 14
Additions 530
Deletions 91

Top hotspots

Path Status +/-
scripts/capture.py added +243 / -0
README.md modified +98 / -90
scripts/fixtures/capture.json added +93 / -0
tests/test_capture.py added +45 / -0
.github/workflows/tests.yml added +24 / -0

Top directories

Directory Files Total changes
scripts 1 243
. 2 191
scripts/fixtures 1 93
tests 1 45
.github/workflows 1 24
Analysis Depth Readiness (evidence-backed, 50%)

ECC Tools uses this to decide whether recommendations should stay at commit-history/setup guidance or expand into CI, security, harness, reference-set, AI-routing, and team backlog work.

Area Status Evidence / Next Step
Commit history Partial 1 commits sampled
CI/CD signals Ready .github/workflows/tests.yml
Security evidence Missing Add AgentShield, audit, SARIF, SBOM, or security review evidence so recommendations can cover security posture.
Harness configuration Missing Add Claude, Codex, OpenCode, Zed, dmux, MCP, plugin, or cross-harness config evidence for harness-agnostic recommendations.
Reference/eval evidence Ready scripts/fixtures/capture.json
AI routing and cost controls Ready docs/03-cost.png
Team handoff and project tracking Missing Add roadmap, runbook, project, Linear, or follow-up tracking docs so generated work can land in a team queue.
Reference Set Readiness (0/7, 0%)
Area Status Evidence / Next Step
Deep analyzer corpus Missing Add analyzer fixture, golden, benchmark, or reference-set files that can catch analyzer regressions.
RAG/evaluator comparison Missing Add retrieval or evaluator reference-set comparison fixtures with expected ranking behavior.
PR salvage/review corpus Missing Add stale-PR, review-thread, reopen-flow, or salvage reference cases for queue cleanup automation.
Discussion triage corpus Missing Add public discussion triage fixtures, golden cases, or reference sets for informational, answered, and no-response classifications.
Harness compatibility Missing Add cross-harness, adapter-compliance, or harness-audit evidence for Claude, Codex, OpenCode, Zed, dmux, and agent surfaces.
Security evidence Missing Attach security evidence such as SBOMs, SARIF, audit reports, or AgentShield evidence packs.
CI failure-mode evidence Missing Add captured CI failure logs, dry-run fixtures, or troubleshooting docs for common workflow failure modes.
Likely Future Issues (2)
Severity Signal Why it may show up
MEDIUM CI workflow changes may ship without failure-mode evidence 1 CI/test-runner paths changed; 0 CI failure-mode evidence artifacts changed
MEDIUM Dependency or CI drift could surface after merge CI/workflow files changed; no lockfile changes detected
  • CI workflow changes may ship without failure-mode evidence: The PR changes CI workflows or test-runner entrypoints without touching CI failure fixtures, captured logs, troubleshooting notes, or regression evidence.
  • Dependency or CI drift could surface after merge: Package or workflow changes landed without an accompanying lockfile update, which often turns into CI or release noise later.
Suggested Follow-up Work (2)
Type Suggested title Targets
PR ci: add failure-mode evidence for .github/workflows/tests.yml .github/workflows/tests.yml
PR chore: refresh lockfile and validate CI after dependency updates .github/workflows/tests.yml
  • ci: add failure-mode evidence for .github/workflows/tests.yml: Backfill CI failure-mode evidence before another workflow or test-runner change lands on the touched surface.
  • chore: refresh lockfile and validate CI after dependency updates: Package or workflow changes without a lockfile refresh tend to turn into noisy follow-up fixes after merge.

Copy-ready bodies

ci: add failure-mode evidence for .github/workflows/tests.yml

## Summary
- Add CI failure-mode evidence for the recently changed workflow or test-runner surface.

## Why
- Backfill CI failure-mode evidence before another workflow or test-runner change lands on the touched surface.

## Touched paths
- `.github/workflows/tests.yml`

## Validation
- Add or update a CI failure fixture, captured failing log, troubleshooting note, workflow dry-run evidence, or regression test for the changed CI/test-runner behavior.
- Run the affected workflow or test-runner entrypoint locally or in CI and record pass/fail evidence.

chore: refresh lockfile and validate CI after dependency updates

## Summary
- Refresh the lockfile and rerun CI after the dependency or workflow changes in this PR.

## Why
- Package or workflow changes without a lockfile refresh tend to turn into noisy follow-up fixes after merge.

## Touched paths
- `.github/workflows/tests.yml`

## Validation
- Refresh the lockfile in the same package manager used by the repo.
- Run the repo typecheck / test / CI entrypoints that depend on the updated package graph.
Generated Instincts (6)
Domain Count
git 2
code-style 3
testing 1

After merging, import with:

/instinct-import .claude/homunculus/instincts/inherited/SCAI-instincts.yaml

Files

  • .claude/ecc-tools.json
  • .claude/skills/SCAI/SKILL.md
  • .agents/skills/SCAI/SKILL.md
  • .agents/skills/SCAI/agents/openai.yaml
  • .claude/identity.json
  • .codex/config.toml
  • .codex/AGENTS.md
  • .codex/agents/explorer.toml
  • .codex/agents/reviewer.toml
  • .codex/agents/docs-researcher.toml
  • .claude/homunculus/instincts/inherited/SCAI-instincts.yaml

ECC Tools | Everything Claude Code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant